Skip to content

[CI][Bugfix] Drop duplicated examples/ prefix in tensorize_vllm_model command#42039

Merged
vllm-bot merged 1 commit intovllm-project:mainfrom
haosdent:ci-e790b261
May 8, 2026
Merged

[CI][Bugfix] Drop duplicated examples/ prefix in tensorize_vllm_model command#42039
vllm-bot merged 1 commit intovllm-project:mainfrom
haosdent:ci-e790b261

Conversation

@haosdent
Copy link
Copy Markdown
Contributor

@haosdent haosdent commented May 8, 2026

Purpose

The "Examples" steps run with working_dir: /vllm-workspace/examples, but #41084 left a redundant examples/ prefix in the tensorize_vllm_model.py command at 5 sites across 3 buildkite YAMLs, resolving to /vllm-workspace/examples/examples/features/... and failing both the Model Runner V2 Examples step and the Examples step. Drop the prefix to match the bare-relative convention used by neighbouring commands.

Test Plan

Test Result

… command

The Examples steps run with working_dir: /vllm-workspace/examples, but the
serialize/deserialize one-liner was invoking
`python3 examples/features/tensorize_vllm_model.py ...`, which resolved to
`/vllm-workspace/examples/examples/features/...` and failed with
"No such file or directory" in build #65099 ("Model Runner V2 Examples").

The redundant `examples/` prefix was introduced by PR vllm-project#41084
([Examples][last/6] Resettle examples) and copy-pasted into five sites:
- .buildkite/test_areas/model_runner_v2.yaml (NV)
- .buildkite/test_areas/misc.yaml (NV)
- .buildkite/test-amd.yaml (mi250 / mi300 / mi355)

Drop the prefix so the command matches the bare-relative convention used by
every neighbouring command in the same steps.

Signed-off-by: haosdent <haosdent@hotmail.com>

Signed-off-by: haosdent <haosdent@gmail.com>
@mergify mergify Bot added ci/build bug Something isn't working labels May 8, 2026
@haosdent haosdent changed the title [WIP] [CI][Bugfix] Drop duplicated examples/ prefix in tensorize_vllm_model command [CI][Bugfix] Drop duplicated examples/ prefix in tensorize_vllm_model command May 8, 2026
@haosdent
Copy link
Copy Markdown
Contributor Author

haosdent commented May 8, 2026

cc @DarkLight1337 @noooop

@haosdent haosdent marked this pull request as ready for review May 8, 2026 07:31
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the file path for the tensorize_vllm_model.py script in several Buildkite configuration files, moving it from examples/features/ to features/. I have no feedback to provide.

Copy link
Copy Markdown
Collaborator

@noooop noooop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

@noooop noooop enabled auto-merge (squash) May 8, 2026 07:33
@github-actions github-actions Bot added the ready ONLY add when PR is ready to merge/full CI is needed label May 8, 2026
@vllm-bot vllm-bot merged commit 36b2c79 into vllm-project:main May 8, 2026
32 of 35 checks passed
libinta pushed a commit to libinta/vllm that referenced this pull request May 8, 2026
… command (vllm-project#42039)

Signed-off-by: haosdent <haosdent@gmail.com>
Signed-off-by: Libin Tang <libin.tang@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci/build ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants